Skip to content

[Feat] 스터디 후기#124

Merged
aken-you merged 37 commits intodevelopfrom
feat/study-review
Aug 26, 2025
Merged

[Feat] 스터디 후기#124
aken-you merged 37 commits intodevelopfrom
feat/study-review

Conversation

@aken-you
Copy link
Contributor

@aken-you aken-you commented Aug 24, 2025

☘️ 작업 내용

민주님... 저도 #121 이 작업과 스터디 후기 작업이 합쳐졌습니다...
죄송합니다 🙇

코드 양이 많아서 미리 올립니다.

구현이 안된 부분부터 설명드릴게요. 스터디 후기 기능 자체는 지금 현재 테스트하기 어려우실 수 있는데요.
(평일에는 잘 조회되던 api에서 에러 응답이 와서, 여쭤본 상태입니다.)

1. 스터디 후기 모달 렌더링

image

2. 후기 작성 완료 모달

image

- 피드백 링크는 나오면 추가할 예정
- 구현되어있지 않으므로 주석처리
@aken-you aken-you self-assigned this Aug 24, 2025
@aken-you aken-you closed this Aug 24, 2025
@aken-you aken-you reopened this Aug 24, 2025
Comment on lines +195 to +197
--color-background-neutral-subtle: var(--color-gray-200);
--color-background-neutral-default: var(--color-gray-500);
--color-background-neutral-strong: var(--color-gray-900);
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

이것들은 혹시 새로 디자인 시스템에 추가된 건가요??
만약 이번에 추가된 게 아니라 원래 있던 건데 등록되어 있지 않았던거라면, 디자인 시스템도 나중에 한 번 쭉 수정해야 하긴 해야겠네요....😥

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

원래 디자인에는 있었는데, 이 파일에 없어서 추가했습니다!

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

혹시 이건 뭔가요....?
이미지가 보이지 않아서...어디에 쓰는 녀석인가요?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

체크박스에서 체크 아이콘입니다

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

혹시 이 파일 한 번만 확인해주실 수 있나요?
원래 my-page에 있던 녀석인데, 아무래도 유저 관련으로 공통적으로 쓰이게 될 UI 같아서 entity로 옮겼습니다...
이 과정에서 좀 문제가 있던 건지 feature과 entity 두 곳에 모달이 존재하는 것 같아요

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

두 곳에 있네요! feature에 있는 모달 삭제하겠습니다.

Comment on lines +141 to +145
export interface UserPositiveKeywordsResponse {
totalCount: number | null; // params에 pageSize 값을 보내지 않는 경우 null
reviewerCount: number | null; // params에 pageSize 값을 보내지 않는 경우 null
keywords: Keyword[];
}
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

이거 저도 좀 고민했던 방식인데 저희가 null 값을 보낼 경우 통합 형식을 한 번 정하면 좋을 것 같아요.
totalCount: number | null; 이런 식으로 null을 명시해주면 이게 null을 포함하는 걸 한 눈에 알 수 있어서 좋긴 한데
저는 조금 긴가? 싶어서 totalCount?: number; 이런 형식으로 통합해서 작성했었거든요.

두 방법 중에 저희가 편한 방식 정해서 그걸로 전체 통일하면 좋을 것 같습니다!

@Mimiminz
Copy link
Contributor

민주님... 저도 #121 이 작업과 스터디 후기 작업이 합쳐졌습니다...

ㅋㅋㅋㅋ 아무래도 저희 둘 다 작업이 섞이다보니 이게.......헷갈릴 수 밖에 없는 것 같습니다....🤦
완료되고 슬랙 언급 한 번 주시면 다시 확인하겠습니다!

@aken-you aken-you marked this pull request as ready for review August 26, 2025 15:05
@aken-you aken-you merged commit 1490341 into develop Aug 26, 2025
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants